Skip to content

Conversation

@boekkooi-impossiblecloud

Summary

The go-yaml project was archived on Apr 1, 2025 and is no longer maintained. Luckily the official YAML organization forked the project and is maintaining it under https://github.com/yaml/go-yaml/tree/v3.

Changes

Replaced all occurrences of gopkg.in/yaml.v3 with go.yaml.in/yaml/v3

Motivation

go-yaml was archived on Apr 1.

Related issues

The [go-yaml](https://github.com/go-yaml/yaml/) project was archived on Apr 1, 2025 and is no longer maintained.
Luckily the official YAML organization forked the project and is maintaining it under https://github.com/yaml/go-yaml/tree/v3.
@dolmen
Copy link
Collaborator

dolmen commented Aug 1, 2025

There is work in progress about the license of that project: yaml/go-yaml#6

So we should block until that is stabilized.

@harryzcy
Copy link

There is work in progress about the license of that project: yaml/go-yaml#6

So we should block until that is stabilized.

License of go-yaml has changed to Apache now. This PR can move forward

@ccoVeille
Copy link
Collaborator

The change are only on main branch for now.

go-yaml has only released -rc for now for the v4.

It might be a bit early

@dolmen dolmen added YAML About YAML and dependency pkg-assert Change related to package testify/assert pkg-require Change related to package testify/require dependencies Pull requests that update a dependency file labels Sep 2, 2025
@mrideout
Copy link

I'd like to see this PR merged, with the v3 tag it now has, while the YAML organization works on v4. That move would make it easier to get buy-in for using Testify at organizations that balk at running abandonware. The main branch's README received a July 30 update that makes it clear that v3 will receive security updates.

Copy link

@fredbi fredbi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The change look good to me

@fredbi
Copy link

fredbi commented Sep 20, 2025

I think this is good to be merged, so instantly all indirect dependencies to the archive repo would disappear.
Notice that the build tags trick that is used in assert/yaml has no influence on how go.mod propagates dependencies.

fredbi added a commit to fredbi/swag that referenced this pull request Sep 20, 2025
* fixes go-openapi#127

The direct dependencies to this archived repo have been replaced by the active
fork imported from go.yaml.in/yaml/v3.

NOTE: an indirect dependency remains from stretchr/testify.
There is an open PR there to solve this stretchr/testify#1772.
There is not much I can do here to remove this indirect test dependency.

Signed-off-by: Frederic BIDON <[email protected]>
fredbi added a commit to go-openapi/swag that referenced this pull request Sep 20, 2025
* fixes #127

The direct dependencies to this archived repo have been replaced by the active
fork imported from go.yaml.in/yaml/v3.

NOTE: an indirect dependency remains from stretchr/testify.
There is an open PR there to solve this stretchr/testify#1772.
There is not much I can do here to remove this indirect test dependency.

Signed-off-by: Frederic BIDON <[email protected]>
@brackendawson
Copy link
Collaborator

The majority of testify users would need to wait until we make a release to benefit from this. Recently we've been on a (very) approximate 6-monthly release cadence, that's not to say we couldn't release earlier than expected to mitigate this and another EOL dependency.

My point is that there isn't a pressing need to rush this. We really do want to merge this, but v4 is in release candidate so I don't think it will be all that long now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file pkg-assert Change related to package testify/assert pkg-require Change related to package testify/require YAML About YAML and dependency

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants